Skip to content

Add sample design pic#9

Merged
shuo-zhou merged 1 commit intomainfrom
update-readme
Sep 15, 2025
Merged

Add sample design pic#9
shuo-zhou merged 1 commit intomainfrom
update-readme

Conversation

@shuo-zhou
Copy link
Copy Markdown
Member

Pull Request Template

Summary

Add sample design pic


Data Modalities Covered

List the modalities included in this PR (tick all that apply):

  • Image
  • Text
  • Signals
  • Tabular
  • Other

Dataset & DataLoader Design

  • Describe the design decisions for your Dataset and DataLoader implementation.
  • Explain how your design addresses scalability, flexibility, and modularity.
  • Highlight any specific challenges and how you resolved them.

Handling Relationships Between Samples

  • Explain how your DataLoader takes relationships across samples into account. For example: linking samples from different modalities that come from the same subject.
  • Describe how missing modalities or unaligned samples are handled.

Sample Usage Instructions

Provide example code showing how to use your Dataset/DataLoader:

from load_data.dataset import CXRDataset, ECGDataset, MultimodalDataloader

cxr_dataset = CXRDataset(data_dir='path/to/cxr', ...)
ecg_dataset = ECGDataset(data_dir='path/to/ecg', ...)
dataset_list = [cxr_dataset, ecg_dataset]
loader = MultimodalDataloader(dataset_list, batch_size=8, shuffle=True)

for batch in loader:
    print(batch.keys())

Extensibility

Explain how this implementation can be extended to support additional modalities.

  • What should be modified or subclassed?
  • What conventions should be followed?

Checklist

  • Code runs without errors
  • Code is well-documented
  • Tests are included and pass
  • Follows project coding standards

Additional Notes

Any additional information or context that reviewers should be aware of.

@shuo-zhou shuo-zhou requested a review from wenruifan September 15, 2025 15:06
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.77%. Comparing base (c63d6a9) to head (7dd35d1).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   80.77%   80.77%           
=======================================
  Files          10       10           
  Lines         593      593           
=======================================
  Hits          479      479           
  Misses        114      114           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shuo-zhou shuo-zhou merged commit f06069b into main Sep 15, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants